fix: route datamate tools through local stdio mcp-engine instead of cloud by altimate-harness-bot[bot] · Pull Request #893 · AltimateAI/altimate-code

altimate-harness-bot · 2026-06-05T06:30:45Z

Problem

DatamateManagerTool.handleAdd() was creating MCP server entries pointing to the cloud MCP server (https://mcpserver.getaltimate.com) because buildMcpConfig() always fell back to DEFAULT_MCP_URL when mcpServerUrl was absent from altimate.json.

The cloud MCP runs in multi-user mode and reads connections from API headers — it has no access to ~/.altimate/connections.json. So any tool call like datamate-remote_jira_get_issue failed with "No connection configured for integration: jira" even when connections were configured locally.

Fix

`datamate.ts` — `handleAdd()`

Replace buildMcpConfig() (HTTP remote) with a local stdio config:

const mcpConfig = {
  type: "local" as const,
  command: ["datamate", "start-stdio", "--datamate", args.datamate_id],
}

Each spawned datamate start-stdio process:

Reads connections.json from ~/.altimate/ directly (local engine, same as VS Code extension)
Discovers the right VS Code extension bridge at runtime via cwd-based sidecar matching in ~/.altimate/extension-rpc/
Multiple VS Code windows are handled correctly — each window's bridge writes a sidecar with its workspaceFolders; the spawned process matches on process.cwd() (set to project root by StdioClientTransport) and routes to the right bridge automatically

`mcp-discover.ts` — `stripSessionEnv()`

When persisting a discovered server via mcp_discover, strip ALTIMATE_EXTENSION_RPC from the environment block before writing to disk. The socket path is session-specific (unique to the current extension host process) — hardcoding it would cause future sessions to connect to a dead or wrong bridge. Stripping it forces runtime discovery.

Prerequisites

Requires the global datamate CLI to be on PATH: npm install -g @altimateai/datamate. The VS Code extension already warns users when stdio mode is configured but no global datamate is found.

Related PRs

AltimateAI/altimate-mcp-engine#211 — writes mcpServerUrl to altimate.json (interim fix, superseded by this approach)
AltimateAI/vscode-altimate-mcp-server#360 — same interim fix for the VS Code extension

Requested by @saravmajestic via harness

Summary by cubic

Route datamate tools through the local MCP engine when an IDE MCP entry exists, auto-detecting stdio vs HTTP from IDE configs and reusing the IDE’s process. Consolidates tools behind a single datamate gateway, adds live reload, and keeps config and enabled state in sync across restarts.

Bug Fixes
- datamate.ts: detect transport from .vscode/mcp.json, .cursor/mcp.json, .github/copilot/mcp.json; reuse IDE command; use server name datamate when an IDE entry exists, otherwise fall back to per-datamate cloud config; persist enabled: true; reconnect existing entries with MCP.connect to preserve enabled state; avoids duplicate stdio processes.
- altimate/datamate-transport.ts: new helpers (DATAMATE_KEY, readDatamateTransportFromIde, syncDatamateUrlFromVscodeMcp, readMcpEntryFromDisk); sync the datamate entry by updatedAt (covers stdio + HTTP); URL-compare for other remote entries.
- Local engine uses ~/.altimate/connections.json and cwd-matched sidecars to route to the right VS Code bridge; supports multiple windows.
- mcp-discover.ts: strip ALTIMATE_EXTENSION_RPC before persisting to avoid stale/dead sockets.
- serve.ts + config.ts: sync datamate from IDE MCP config on startup; persist updatedAt in config.
- server.ts: add POST /altimate/mcp/reload-datamate; bypass stale in-memory config by reading fresh entries from disk and reconnect with MCP.add (no restart, no enabled flag clobber).
- session/prompt.ts: subscribe to MCP.ToolsChanged and refresh tools on the next turn with trace logs.
- mcp/index.ts: persist enabled/disabled to config on connect/disconnect so it survives restarts.
- command/index.ts: add /mcps command to list MCP servers and status (AI-6948).
Migration
- Ensure the datamate CLI is on PATH: npm install -g @altimateai/datamate.

^{Written for commit 520143f. Summary will update on new commits.}

github-actions · 2026-06-05T06:30:56Z

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

github-actions · 2026-06-05T06:30:57Z

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

Open an issue describing the bug/feature (if one doesn't exist)
Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

github-actions · 2026-06-05T06:31:19Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

altimate-harness-bot · 2026-06-05T07:32:25Z

E2E Verification — PASSED ✓

Full pipeline tested end-to-end in the sandbox:

Test setup:

altimate serve running on port 4097 (bun source, live-reloads PR branch changes)
Mock Altimate backend on port 5001 (returns Jira datamate with type: "tool")
Mock Jira REST API on port 5001 (serves /rest/api/2/issue/AI-6348)
connections.json using type: "server" + jiraBaseUrl: "http://localhost:5001" (avoids need for real credentials)

Results:

✓ Initialized: AltimateMCPStdioServer
✓ Tools: 8 total, 8 Jira tools
   - jira_get_issue, jira_create_issue, jira_get_child_issues,
     jira_add_comment, jira_update_issue_status, jira_get_available_transitions,
     jira_link_issue, jira_get_link_types

→ Calling jira_get_issue(issueIdOrKey: "AI-6348")...

✓ jira_get_issue("AI-6348") SUCCESS!
  key: AI-6348
  summary: feat: Ingest Snowflake TASK_HISTORY / SERVERLESS_TASK_HISTORY / COMPLETE_TASK_GRAPHS...
  status: Done
  assignee: Raghwendra Singh

What was verified:

datamate.ts patch: handleAdd() creates { type: "local", command: ["datamate", "start-stdio", "--datamate", id] } instead of cloud HTTP config
mcp-discover.ts patch: stripSessionEnv() removes ALTIMATE_EXTENSION_RPC before persisting to altimate-code.json — verified environment key absent in written config
Full MCP JSON-RPC stdio handshake: initialize → tools/list (8 tools) → tools/call → real tool response
discoverExternalMcp reads .vscode/mcp.json correctly, transforms stdio entry to { type: "local" }, strips session socket path

github-actions · 2026-06-05T09:43:41Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-05T09:44:12Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-05T10:02:53Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-05T10:03:16Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-05T10:23:25Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-05T10:23:50Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-05T13:03:35Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-05T13:04:05Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-05T13:20:25Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-05T13:20:53Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-05T13:31:42Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-05T13:32:20Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-05T13:53:04Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-05T13:53:49Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-09T01:45:39Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-09T01:46:01Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

- Add stdio transport support for datamate MCP (vs HTTP-only before) - Single-gateway mode: when .vscode/mcp.json has "datamate" key, always use it as server name — prevents duplicate tool sets from extension - syncDatamateUrlFromVscodeMcp: use updatedAt field as change signal for the "datamate" entry (works for both stdio and HTTP), URL comparison for all other remote entries - Strip ALTIMATE_EXTENSION_RPC from persisted mcp-discover configs to avoid stale socket paths across VS Code sessions - persistMcpEnabled: write enabled/disabled flag to disk on MCP connect/disconnect so it survives session restarts - Add /altimate/mcp/reload-datamate endpoint to re-sync and reconnect without full server restart - MCP.ToolsChanged subscription in prompt loop for traceability - Merge main: preserve trace consumer in serve.ts, restore exports on isAnthropicLikeModel and insertReminders

github-actions · 2026-06-09T02:46:23Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-09T02:47:00Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

) Co-authored-by: saravmajestic <[email protected]>

github-actions · 2026-06-11T02:31:51Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-11T02:32:26Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

saravmajestic · 2026-06-11T09:19:45Z

+  projectRootDir: string,
+): Promise<{ type: "remote"; url: string } | { type: "local" } | null> {
+  try {
+    const mcpJsonPath = path.join(projectRootDir, ".vscode", "mcp.json")


this will not be in .vscode always, it will differ according to the IDE used. We should scan across projectRootDir

saravmajestic · 2026-06-11T09:21:22Z

    "global config is at ~/.config/altimate-code/altimate-code.json. " +
-    "Datamate server names are prefixed with 'datamate-'. " +
+    "When a VS Code extension datamate entry exists (.vscode/mcp.json has 'datamate' key), " +
+    "'add' always uses the server name 'datamate' — tools are then prefixed 'datamate_'. " +


what is referred as add here?

saravmajestic · 2026-06-11T09:23:27Z

+    for (const [key, entry] of Object.entries(serversMap)) {
+      const args = Array.isArray(entry["args"]) ? (entry["args"] as string[]) : []
+      const isDatamate =
+        key === "datamate" ||


can use EXTENSION_DATAMATE_SERVER instead of string?

saravmajestic · 2026-06-11T09:25:22Z

+        : transport?.type === "local"
+          // Extension stdio: no --datamate id needed — active teammate is resolved
+          // by the extension over the ALTIMATE_EXTENSION_RPC socket at runtime.
+          ? { type: "local" as const, command: ["datamate", "start-stdio"] }


isnt this supposed to be same as in .vscode/mcp.json, so that it can re use the same process? Otherwise we will be maintaining 2 processes?

saravmajestic · 2026-06-11T09:28:15Z

+// Stripping it forces runtime discovery via ~/.altimate/extension-rpc/ sidecars,
+// which always resolves the correct live bridge by matching process.cwd() against
+// each bridge's recorded workspaceFolders.
+function stripSessionEnv(cfg: import("../../config/config").Config.Mcp): import("../../config/config").Config.Mcp {


Ideal case is to use same process. May be we can expose a cleanup function, which can be called by extension on deactivate or window close? Or actually, when window is closed, this altimate code process will also be killed right?

saravmajestic · 2026-06-11T09:28:51Z

 import { subscribeTraceConsumer } from "../../altimate/observability/trace-consumer"
 // altimate_change end

+// altimate_change start


this logic could better be in separate file

saravmajestic · 2026-06-11T09:29:07Z

+// All other remote MCP entries fall back to URL comparison (original behaviour).
+// Fire-and-forget: errors are logged but never thrown.
+// Returns the list of MCP server names whose config was updated.
+const DATAMATE_KEY = "datamate"


repeated constant. lets extract to common constant

saravmajestic · 2026-06-11T09:29:48Z

+export async function syncDatamateUrlFromVscodeMcp(cwd: string): Promise<string[]> {
+  const updated: string[] = []
+  try {
+    const mcpJsonPath = path.join(cwd, ".vscode", "mcp.json")


seems to be repeated logic

saravmajestic · 2026-06-11T09:31:50Z

  // check would miss the gateway-emitted specific names (#888 J1). The api.id
  // checks are lowercased and tightened to a `claude-` / `anthropic-` /
  // `anthropic/...` shape so a model named `foo-claude-bench` doesn't false-match.
-  //


why is this removed? same for below comment block

…+ improve logging

github-actions · 2026-06-11T09:46:18Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

github-actions · 2026-06-11T09:46:49Z

👋 This PR was automatically closed by our quality checks.

Common reasons:

New GitHub account with limited contribution history
PR description doesn't meet our guidelines
Contribution appears to be AI-generated without meaningful review

If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you.

altimate-harness-bot Bot assigned saravmajestic Jun 5, 2026

github-actions Bot added needs:compliance contributor labels Jun 5, 2026

github-actions Bot added the needs:issue label Jun 5, 2026

github-actions Bot added the needs-review:blocked label Jun 5, 2026

altimate-harness-bot Bot force-pushed the fix/datamate-stdio-local-transport branch from a71092d to 0b6cc26 Compare June 9, 2026 02:46

feat: [AI-6948] register mcps slash command in server command list (#885

d8c851f

) Co-authored-by: saravmajestic <[email protected]>

saravmajestic reviewed Jun 11, 2026

View reviewed changes

fix: address review comments + stale config cache in reload-datamate …

520143f

…+ improve logging

Conversation

altimate-harness-bot Bot commented Jun 5, 2026 • edited by cubic-dev-ai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Problem

Fix

datamate.ts — handleAdd()

mcp-discover.ts — stripSessionEnv()

Prerequisites

Related PRs

Summary by cubic

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

altimate-harness-bot Bot commented Jun 5, 2026

E2E Verification — PASSED ✓

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 5, 2026

Uh oh!

github-actions Bot commented Jun 9, 2026

Uh oh!

github-actions Bot commented Jun 9, 2026

Uh oh!

github-actions Bot commented Jun 9, 2026

Uh oh!

github-actions Bot commented Jun 9, 2026

Uh oh!

github-actions Bot commented Jun 11, 2026

Uh oh!

github-actions Bot commented Jun 11, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Jun 11, 2026

Uh oh!

altimate-harness-bot Bot commented Jun 5, 2026 •

edited by cubic-dev-ai Bot

Loading

`datamate.ts` — `handleAdd()`

`mcp-discover.ts` — `stripSessionEnv()`